Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed incorrect install of pugixml in dynamic build #8566

Merged
merged 1 commit into from
Nov 12, 2021

Conversation

ilya-lavrenov
Copy link
Contributor

Details:

  • ov_install_static_lib relies on BUILD_SHARED_LIBS. So, we need to put BUILD_SHARED_LIBS to different scope of visbility.

Tickets:

@ilya-lavrenov ilya-lavrenov added the category: build OpenVINO cmake script / infra label Nov 12, 2021
@ilya-lavrenov ilya-lavrenov added this to the 2022.1 milestone Nov 12, 2021
@ilya-lavrenov ilya-lavrenov self-assigned this Nov 12, 2021
@ilya-lavrenov ilya-lavrenov requested a review from a team November 12, 2021 16:18
@openvino-pushbot openvino-pushbot added the category: inference OpenVINO Runtime library - Inference label Nov 12, 2021
@ilya-lavrenov ilya-lavrenov enabled auto-merge (squash) November 12, 2021 16:37
@ilya-lavrenov ilya-lavrenov merged commit a0b5de0 into openvinotoolkit:master Nov 12, 2021
@ilya-lavrenov ilya-lavrenov deleted the pugixml-install branch November 13, 2021 11:16
openvino-dev-samples pushed a commit to openvino-dev-samples/openvino that referenced this pull request Nov 23, 2021
openvino-dev-samples pushed a commit to openvino-dev-samples/openvino that referenced this pull request Nov 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: build OpenVINO cmake script / infra category: inference OpenVINO Runtime library - Inference
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants